Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose EditorExport through the EditorInterface singleton #98251

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

allenwp
Copy link
Contributor

@allenwp allenwp commented Oct 16, 2024

Exposed EditorExport through the EditorInterface singleton. Also exposed functions relating to feature tags of platforms and presets.

Background
Retrieving feature tags in a way similar to the export process is needed to implement my "Resource Remaps" proposal as an EditorExportPlugin. This PR exposes the parts of EditorExport that are required to do this.

Related PRs
Export scripting functionality: #90782
Feature tags in scripting: #86747

Implementation
I have kept the scripting API similar to the C++ API, but that does mean there are some style differences between getting custom features (existing function that returns a comma delimited String) and getting the platform/preset features (new functions that return a TypedArray<String>).

In addition to the functions I've exposed in this PR, the currently protected EditorExportPlatform::get_features function could be exposed to give developers a feature list that is true to the final list of features used during the export process. I did not include this function only because it is currently protected, but if it others believe it should be changed to public, then it would be nice to expose it.

@allenwp allenwp requested review from a team as code owners October 16, 2024 21:25
Also exposed functions relating to feature tags of platforms and presets.
@allenwp allenwp force-pushed the Expose-EditorExport-through-EditorInterface branch from 13e996f to e501c12 Compare October 16, 2024 22:03
@fire fire requested review from a team and removed request for a team October 17, 2024 04:26
@Chaosus Chaosus added this to the 4.4 milestone Oct 17, 2024
@AThousandShips AThousandShips changed the title Exposed EditorExport through the EditorInterface singleton Expose EditorExport through the EditorInterface singleton Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants